Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Excitatory/Inhibitory LIF neuron model #888

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

monkin77
Copy link
Contributor

@monkin77 monkin77 commented Aug 24, 2024

Issue Number: #887

Objective of pull request: Model E/I LIF neurons to represent more complex behaviors containing positive and negative inputs, each with their own decay time constants (du_exc and du_inh). It also allows varying the voltage and current time constants of neurons belonging to the same layer.

Pull request checklist

Your PR fulfills the following requirements:

  • Issue created that explains the change and why it's needed
  • Tests are part of the PR (for bug fixes / features)
  • Docs reviewed and added / updated if needed (for bug fixes / features)
  • PR conforms to Coding Conventions
  • PR applys BSD 3-clause or LGPL2.1+ Licenses to all code files
  • Lint (flakeheaven lint src/lava tests/) and (bandit -r src/lava/.) pass locally
  • Build tests (pytest) passes locally

Pull request type

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation changes
  • Other (please describe):

What is the new behavior?

  • A new neuron model that allows playing with excitatory and inhibitory inputs and the variation between their underlying current time constants (du_exc and du_inh) to represent more complex scenarios

Does this introduce a breaking change?

  • Yes
  • No

Supplemental information

This only contains the floating-point precision version of the model. I believe it is possible to implement a fixed-precision version of it, although it requires further investigation.

I would appreciate your opinion on this as I've not delved deep into the microcode documentation.

@monkin77 monkin77 marked this pull request as ready for review August 24, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant